Skip to content

test_lightningd.py: more minor flake8 fixes. - #1111

Merged
cdecker merged 1 commit into
ElementsProject:masterfrom
rustyrussell:unflake8
Feb 26, 2018
Merged

test_lightningd.py: more minor flake8 fixes.#1111
cdecker merged 1 commit into
ElementsProject:masterfrom
rustyrussell:unflake8

Conversation

@rustyrussell

Copy link
Copy Markdown
Contributor

Looks like rebasing the flake8 branch caused breakage, as new violations had occurred since that check was written.

@practicalswift

Copy link
Copy Markdown
Contributor

utACK 85bb41d

Comment thread tests/test_lightningd.py Outdated

# Renable when the bitcoin wallet can generate a "bcrt1" address
#waddr = l1.bitcoin.rpc.getnewaddress()
# #waddr = l1.bitcoin.rpc.getnewaddress()

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The second # is not needed :-)

Comment thread tests/test_lightningd.py Outdated
# Renable when the bitcoin wallet can generate a "bcrt1" address
#assert(len(withdrawal) == 1)
#assert(withdrawal[0]['amount'] == Decimal('0.02'))
# #unspent = l1.bitcoin.rpc.listunspent(0)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here

Looks like rebasing the flake8 branch caused breakage, as new violations
had occurred since that check was written

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
@cdecker

cdecker commented Feb 26, 2018

Copy link
Copy Markdown
Member

ACK 9830def

@cdecker
cdecker merged commit 58fae47 into ElementsProject:master Feb 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants